Add log-type bubble to file browser rows#196
Merged
Merged
Conversation
The browser shows each session by date/time, so the file format wasn't visible anywhere. Add a small type pill (Dove, Dovex, XRK, XRZ, iRacing, VBO, MoTeC, UBX, NMEA, CSV, …) derived from the file extension — we don't persist the detected format, so the extension is the source of truth. - lib/logFileType.ts: pure extension → label mapping (unit-tested). - components/FileTypeBadge.tsx: reusable pill, renders nothing without an extension. - Wired into FilesTab local + cloud rows and the Profile Cloud logs panel.
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
dovesdataviewer | 9b9af1b | Commit Preview URL Branch Preview URL |
Jun 13 2026, 05:54 AM |
Coverage SummaryLines: 58.45% (5209/8911) · Statements: 57.49% · Functions: 55.27% · Branches: 52.67% Per-file coverage
|
The 2.5.1 beta version was already cut, so these changes belong under it rather than a new [Unreleased] block. Merge the entries into ## [2.5.1] - unreleased and add a CLAUDE.md note: once a beta version is picked, keep it as the heading marked "unreleased" and accumulate changes there — no per-commit patch bumps.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
The file browser shows each session by date/time, so there was no way to tell what kind of log a row is. This adds a small type bubble to each row — and since we don't persist the detected format, the file's extension is the source of truth (exactly as suggested).
Changes
lib/logFileType.ts(pure, unit-tested) —logFileExtension()+logFileTypeLabel(): maps an extension to a short label (Dove, Dovex, XRK, XRZ, iRacing, VBO, MoTeC, UBX, NMEA, CSV, TXT), uppercases unknown extensions, and returns""when there's no extension.components/FileTypeBadge.tsx— a tiny reusable pill (muted tokens) that renders nothing when the name has no extension.No color/token changes; the bubble uses existing
mutedtokens.Checks
npm run lint✅npm run typecheck✅npm run test:run✅ (110 files, 1504 tests — +7 new forlogFileType)npm run build✅CHANGELOG (
[Unreleased]→ Added) and CLAUDE.md (File Browser section) updated.https://claude.ai/code/session_014P9qC1xKvJUajTTdHuAh7P
Generated by Claude Code